Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(IDX): remove CI github email #3837

Merged
merged 17 commits into from
Feb 11, 2025
Merged

fix(IDX): remove CI github email #3837

merged 17 commits into from
Feb 11, 2025

Conversation

cgundy
Copy link
Member

@cgundy cgundy commented Feb 6, 2025

For any auto-generated commits we need to add both a user and an email, though this user is not actually logged in. This can cause confusion if the email address is associated with a real user that doesn't have anything to do with this commit.

The ideal solution would be to set github-actions as the commit author, but I have not figured out a way to do this. For this reason, we just leave the email blank.

@cgundy cgundy changed the title remove github user fix(IDX): remove CI github email Feb 11, 2025
@github-actions github-actions bot added the fix label Feb 11, 2025
@cgundy cgundy marked this pull request as ready for review February 11, 2025 15:54
@cgundy cgundy requested review from a team as code owners February 11, 2025 15:54
@cgundy cgundy added this pull request to the merge queue Feb 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 11, 2025
@cgundy cgundy added this pull request to the merge queue Feb 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 11, 2025
@cgundy cgundy added this pull request to the merge queue Feb 11, 2025
Merged via the queue into master with commit 8c8be0e Feb 11, 2025
35 of 37 checks passed
@cgundy cgundy deleted the remove-github-user branch February 11, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants